fix(cli): avoid node util styleText dependency in setup wizard#12
Closed
freelanceagent1 wants to merge 2 commits intocortexkit:mainfrom
Closed
fix(cli): avoid node util styleText dependency in setup wizard#12freelanceagent1 wants to merge 2 commits intocortexkit:mainfrom
freelanceagent1 wants to merge 2 commits intocortexkit:mainfrom
Conversation
freelanceagent1
force-pushed
the
fix/cli-node19-prompts
branch
from
7215f95 to
07cefc0
Compare
Collaborator
|
Thanks for the PR and the careful diagnosis — you correctly identified the root cause. We ended up fixing this a different way in v0.13.1: instead of removing `@clack/prompts`, we changed all documented commands to use `bunx --bun` instead of `bunx`. The `--bun` flag tells bun to execute the JavaScript directly with its own runtime, bypassing the `#!/usr/bin/env node` shebang that was routing to the user's older Node. This preserves the `@clack/prompts` UX (colors, prompts, spinners) while working on any Node version — or with no Node installed at all. Closing this PR since the underlying issue (#10) is resolved, but genuinely appreciate the contribution and investigation. 🙏 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Fixes the
aft-opencode setup/doctorCLI crash on older Node runtimes wherenode:util.styleTextis unavailable.The failure reported was:
Cause
The CLI prompt path depended on
@clack/prompts, which pulled in a runtime path that expected newer Nodenode:utilexports than are available on Node19.9.0.Fix
@clack/promptsfrom the plugin package dependenciessetup/doctorCLI behavior intactValidation
bun installbun run buildinpackages/opencode-pluginnode packages/opencode-plugin/dist/cli.jsAdditionally, the built
dist/cli.jsno longer contains:styleTextstripVTControlCharacters@clack/promptsnode:utilCompatibility note
19.9.0case